Skip to content

feat(IDocTags): add XML escape modes#471

Merged
vagenas merged 5 commits intomainfrom
cdata-wrapping
Jan 12, 2026
Merged

feat(IDocTags): add XML escape modes#471
vagenas merged 5 commits intomainfrom
cdata-wrapping

Conversation

@vagenas
Copy link
Member

@vagenas vagenas commented Jan 9, 2026

No description provided.

@vagenas
feat: add XML escape modes
b450aea 
Signed-off-by: Panos Vagenas <pva@zurich.ibm.com>
@dosubot
Copy link

dosubot bot commented Jan 9, 2026 •
edited
Loading

Related Documentation

Checked 7 published document(s) in 0 knowledge base(s). No updates required.

How did I do? Any feedback?  Join Discord

@mergify
Copy link

mergify bot commented Jan 9, 2026 •
edited
Loading

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?:

🟢 Require two reviewer for test updates

Wonderful, this rule succeeded.

When test data is updated, we require two reviewers

  • #approved-reviews-by >= 2

vagenas
vagenas commented Jan 9, 2026
View reviewed changes
vagenas
vagenas commented Jan 9, 2026
View reviewed changes
@vagenas vagenas marked this pull request as draft January 9, 2026 09:05
@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2026 •
edited
Loading

✅ DCO Check Passed

Thanks @vagenas, all your commits are properly signed off. 🎉

@codecov
Copy link

codecov bot commented Jan 9, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 85.71429% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
docling_core/experimental/idoctags.py 85.71% 5 Missing ⚠️

📢 Thoughts on this report? Let us know!

@PeterStaar-IBM PeterStaar-IBM self-requested a review January 9, 2026 09:25
vagenas added 4 commits January 9, 2026 13:11
@vagenas
remove workaround for IDE debugging
744966f 
Signed-off-by: Panos Vagenas <pva@zurich.ibm.com>
@vagenas
fix inline handling, descope entities mode for now
0581c0b 
Signed-off-by: Panos Vagenas <pva@zurich.ibm.com>
@vagenas
fix escaping/CDATA interference with inline groups, formatting, code …
30190c3 
…languages; add underline tag

Signed-off-by: Panos Vagenas <pva@zurich.ibm.com>
@vagenas
minor cleanup, test expansion
4bf38a7 
Signed-off-by: Panos Vagenas <pva@zurich.ibm.com>
@vagenas vagenas marked this pull request as ready for review January 12, 2026 10:06
@vagenas vagenas requested a review from dolfim-ibm January 12, 2026 10:08
PeterStaar-IBM
PeterStaar-IBM approved these changes Jan 12, 2026
View reviewed changes
Copy link
Member

@PeterStaar-IBM PeterStaar-IBM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perfect!

@vagenas vagenas merged commit 0c9c70d into main Jan 12, 2026
13 checks passed
@vagenas vagenas deleted the cdata-wrapping branch January 12, 2026 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PeterStaar-IBM PeterStaar-IBM PeterStaar-IBM approved these changes

@dolfim-ibm dolfim-ibm dolfim-ibm approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@vagenas @PeterStaar-IBM @dolfim-ibm